Conversation
Cambiada terminología (levantar/ bajar la mano en vez de silenciar) Mensaje de levantar la mano puesto de color naranja Cambio de algoritmo de !Mano
…jarMano) a usuarios en concreto o a todos (menos a ellos)
… el que cargamos las extensiones ("comandos") que se encuentran en ./Extensions
…te a la issue 2. Mudado y reajustado el codigo de los comandos (intruduciendo cogs/categorias) del antiguo bot.py, donde estaba todo junto, a un fichero separado (voice_control)
…ionalidad no cambia). Añadida opción para dar la mano a todos (@everyone)
Thurmiel
left a comment
There was a problem hiding this comment.
En la linea 72, donde has puesto : tiene sentido?
Yo creo que no hace falta la variable channel, simplemente usa message.channel, no se utiliza tanto como para hacer una variable, haz el cambio tú @Daniel-Tomas , que yo no puedo desde donde estoy.
En cuanto pueda me pondre ha hacer test.
… el codigo añadiendo variables para mejor su legibilidad
… el codigo añadiendo variables para mejorar su legibilidad
Yo creo que si tiene sentido ya que se utilizan varias veces. He puesto variables si y sólo si se usa más de una vez un objeto. |
…izada la clase Bot(). Hace falta crear un script que instancie dicha clase
|
Esto esta casi terminado no? Respecto al archivo script, estoy de acuerdo, molaria tener un run_bot.py o algo asi que ejecutes y sea simplemente un script que crea todas las clases e inicializa todo lo necesario. |
Si, en principio esta terminado, solo tenemos que probarlo y ver si hacemos alguna mejora/modificación, cambiar la estructura y hacer los tests. |
JustAntoRS
left a comment
There was a problem hiding this comment.
El comentario sobre los strings va en general por todas las sumas de strings que he visto, y tambien hecho en falta documentacion en las funciones, como minimo de que tipo son sus argumentos y que representan
reestructurados los strings
… silenciar y desilenciar
Mejorado el funcionamiento del control de voz. Cambiada la estructura del proyecto para que cada issue se desarrolle, con su conjunto de comandos (mediante cogs/categorias : https://discordpy.readthedocs.io/en/stable/ext/commands/api.html?highlight=command#cogs), en ficheros separados (mediante extensions: https://discordpy.readthedocs.io/en/stable/ext/commands/extensions.html#reloading). En este caso, se ha añadido la categoría VoiceControl y su correspondiente extension, voice_control.